-
-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHP8.4 Support Added #312
base: 2.21.x
Are you sure you want to change the base?
PHP8.4 Support Added #312
Conversation
cod43156
commented
Oct 1, 2024
Q | A |
---|---|
Documentation | yes/no |
Bugfix | yes/no |
BC Break | yes/no |
New Feature | yes/no |
RFC | yes/no |
QA | yes/no |
ec05f53
to
651a732
Compare
Signed-off-by: Raj Mohan <[email protected]>
Signed-off-by: Raj Mohan <[email protected]>
651a732
to
30b6baa
Compare
Signed-off-by: Raj Mohan <[email protected]>
Hi @gsteel, the build is failing as the containers are not starting up correctly. Could you advise if there are any known build issues with this repository? |
Here’s a polite and professional comment you can add to request an update: Hello @gsteel, Just following up on the earlier comment regarding the build issues with this repository. Could you kindly confirm if there's anything pending from my side to assist in resolving this? Please let me know if I can help in any way. Thank you for your time and support! |
When 8.3 was new it had the same problem #293 Looks like mssql hasn't updated for 8.4 yet https://github.com/Microsoft/msphpsql/releases |
404241e
to
6f0b590
Compare
@froschdesign @gsteel Would you be able to provide some guidance on how you'd like me to proceed with this? I'm happy to help in any way I can. |
@cod43156 Thanks for the offer of help. I don't use or know laminas-db very well so please stop pinging me. Hopefully someone else on the @laminas/technical-steering-committee will be able to offer you some guidance. |
@cod43156 please rebase and let see if the CI is now working |